Refactor/#110 remove unused comments logger by zzangzzangguy · Pull Request #114 · PopPool/iOS

zzangzzangguy · 2025-04-16T08:38:43Z

📌 이슈

쓰이지 않는 주석 및 로거 제거 #110

✅ 작업 사항

기타 불필요한 주석과 로그 삭제

0Hooni

전반적으로 안쓰이는 코드들을 제거하는 작업을 하신것 같네요!

몇몇 파일들에 대한 의문이 좀 있어서 코멘트 남겨놨습니다. 답변 주시고 리뷰 재요청 눌러주시면 될 것 같아요!

수고하셨습니다 ㅎㅎ

0Hooni · 2025-04-16T08:54:39Z

.gitignore

 fastlane/Preview.html
 fastlane/screenshots/**/*.png
 fastlane/test_output
+Poppool/Poppool/Infrastructure/*.mobileprovision


혹시 프로비저닝이 프로젝트 폴더 안에 있는건가요?

일단 프로비저닝은 프로젝트 폴더 외부에 있어도 등록가능하기에 폴더에 넣는것보다 외부에 두는것을 권장드립니다!

아마 조만간 모듈화 들어가면 까먹으셨다가 또 프로비저닝 들어가십니다 🥲

외부에 두셨다면 ignore 세팅 지워주시기까지 하면 감사하겠습니다 🙇🏻‍♂️

브랜치 분리 이후 에 작업해둔거군뇨.. 외부에 두도록 하겠습니다 😂

@zzangzzangguy 이번 PR에서 제거해주세요!

이번 PR에 들어가버리면 나중에 까먹고 남아있을것 같아서🥲

0Hooni · 2025-04-16T08:54:57Z

Poppool/File.txt

무슨 파일이죠?

0Hooni · 2025-04-16T08:55:54Z

Poppool/Poppool/Presentation/Scene/Admin/AdminStoreCell.swift

    // MARK: - Configure
    func configure(with store: GetAdminPopUpStoreListResponseDTO.PopUpStore) {
-        Logger.log(message: "셀 데이터 바인딩: \(store)", category: .debug)
+//        Logger.log(message: "셀 데이터 바인딩: \(store)", category: .debug)


사용하지 않는다면 지우시는것 추천드립니다!

0Hooni · 2025-04-16T08:56:01Z

Poppool/Poppool/Presentation/Scene/Admin/AdminStoreCell.swift

-        // mainImageUrl에서 baseURL 부분 제거
        let imagePath = store.mainImageUrl.replacingOccurrences(of: KeyPath.popPoolS3BaseURL, with: "")
-        Logger.log(message: "이미지 경로: \(imagePath)", category: .debug)
+//        Logger.log(message: "이미지 경로: \(imagePath)", category: .debug)


해당화면 디버깅중에 잠깐 주석처리해둔 부분인데 필요치않은것 같아 다음 PR에 제거하겠습니다

@zzangzzangguy 어차피 지금 PR이 주석 제거니까 그냥 여기서 지워도 될것 같아요!

0Hooni · 2025-04-16T08:56:49Z

Poppool/Poppool/Presentation/Scene/Admin/AdminRegister/PopUpStoreRegisterViewController.swift

    private func showLoadingIndicator() {
-        // 로딩 인디케이터 표시 로직 구현
-        // 예: Activity Indicator 또는 커스텀 로딩 뷰 표시
+
    }

    private func hideLoadingIndicator() {
-        // 로딩 인디케이터 숨김 로직 구현
    }


이렇게 되면 비어있는 메서드인데 지우시는건 어떨까요?

레거시코드.. ㅜ 제거하도록 하겠습니다

0Hooni · 2025-04-16T08:59:01Z

Poppool/Poppool/Presentation/Scene/Map/MapView/MapViewController+Handlers.swift

안쓰는 코드는 꼭 삭제....!!!

전체 주석이 아마 나중에 쓰려고 냅두는거같은데, 개인적으로 좋지 않은 습관이라고 생각해요...🥲

사용이 필요하다면 사용할 때 구현하시고, 나중에 쓰시려 하시는 거라면 팀의 레포지토라가 아닌 개인 폴더로 관리하시는것도 좋을것 같네욥☺️

아 기존 해당방식으로 MapViewController 를 분리하려 하다 브랜치분리 이후 제거된 코드인데 올라갔군요 ! 제거되었습니다

@zzangzzangguy 지금 브랜치 뎁스가 어떻게 되어있는건지 알 수 있을까요?

지금 PR에서는 제거된 커밋이 안올라온것 같아요!

0Hooni · 2025-04-16T09:01:47Z

Poppool/Poppool/Presentation/Scene/Map/MapView/MapViewController.swift

        }

        private func showNoMarkersToast() {
-            // 디자인 예정이므로 임시 구현


이런 주석은 일반 주석보다는 TODO 주석을 이용해보는것도 방법입니다!

// TODO: 할 일

이렇게 사용하시면 됩니다👍🏻

dongglehada

영훈님이 상세하게 확인을 해주셔서 저는 코멘트 하나 남긴 부분말고는 확인하였습니다!

dongglehada · 2025-04-16T14:56:41Z

Poppool/Poppool/Presentation/Scene/Map/MapView/MapSearchInput.swift

-                print("[DEBUG] TextField Input: \(text)")
+            .subscribe(onNext: { _ in
            })
            .disposed(by: disposeBag)


해당 옵저버를 사용하지 않는 것 같은데 지워도 괜찮지 않을까요.?

To DO로 변경되었습니다!

zzangzzangguy and others added 6 commits April 15, 2025 08:02

refactor/#110: RegisterVC View와 PopUpImagesColletionView로 분리

ecbffa1

refactor/#110: gitignore 업데이트

fadb4d7

refactor/#110: 임시저장

5c0a2e4

fix/#110: 맵뷰컨트롤러 주석및 로거 정리

1854d5e

refactor/#110: 주석및 로거 제거

293b3cc

style/#110: Apply SwiftLint autocorrect

ae89367

zzangzzangguy assigned 0Hooni and dongglehada Apr 16, 2025

zzangzzangguy added the 🔄 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 label Apr 16, 2025

zzangzzangguy requested review from 0Hooni and dongglehada April 16, 2025 08:52

zzangzzangguy unassigned 0Hooni and dongglehada Apr 16, 2025

0Hooni reviewed Apr 16, 2025

View reviewed changes

0Hooni assigned zzangzzangguy Apr 16, 2025

0Hooni linked an issue Apr 16, 2025 that may be closed by this pull request

쓰이지 않는 주석 및 로거 제거 #110

Closed

dongglehada reviewed Apr 16, 2025

View reviewed changes

github-actions bot added 2 commits April 18, 2025 13:27

style/#110: Apply SwiftLint autocorrect

b1dba60

style/#110: Apply SwiftLint autocorrect

592540b

0Hooni removed a link to an issue Apr 18, 2025

쓰이지 않는 주석 및 로거 제거 #110

Closed

0Hooni closed this Apr 18, 2025

0Hooni deleted the refactor/#110-remove-unused-comments-logger branch October 19, 2025 11:18

Conversation

zzangzzangguy commented Apr 16, 2025

📌 이슈

✅ 작업 사항

Uh oh!

0Hooni left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

0Hooni Apr 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dongglehada left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

0Hooni Apr 16, 2025 •

edited

Loading